Skip to content

[Bug] Capitalize tag-based filter button labels on homepage#20

Merged
nojibe merged 1 commit into
weval-org:mainfrom
nojibe:ken/bug/fix-safety-tag-case
Jun 15, 2026
Merged

[Bug] Capitalize tag-based filter button labels on homepage#20
nojibe merged 1 commit into
weval-org:mainfrom
nojibe:ken/bug/fix-safety-tag-case

Conversation

@nojibe

@nojibe nojibe commented May 5, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes a display bug where tag-based filter buttons on the homepage rendered in lowercase (e.g. "safety" instead of "Safety").

Before:
image

Summary

  • Tag names sourced from blueprint YAML are lowercase strings. EvaluationFilterSection was passing the raw tag name directly as the pill label with no capitalization applied.
  • Adds Tailwind's capitalize utility class to the pill button, consistent with how the rest of the codebase handles this pattern (see MacroCoverageTable, SharedEvaluationComponents). The filter key is unchanged so filtering behaviour is unaffected — display only.

Fixes in-app bug report from nnojibe@gmail.com.

Closes: #17

PR written with assistance from Claude (Anthropic).

Tag names sourced from blueprint YAML are lowercase strings. EvaluationFilterSection
was using the raw tag name as the display label with no capitalization, causing
buttons like "safety" to appear in lowercase.

Adds Tailwind's `capitalize` utility to the pill button className, consistent
with how the rest of the codebase handles this pattern. The filter key is
unchanged so filtering behaviour is unaffected.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@nojibe nojibe changed the title [Bug] capitalize tag-based filter button labels on homepage [Bug] Capitalize tag-based filter button labels on homepage May 5, 2026
@nojibe nojibe merged commit 4c9951d into weval-org:main Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] safety in the filter button is lowercase, when it should be sentence case. Th...

1 participant